Skip to content

Updating workflow to include code validation PHP8.4 and MariaDB 10.6#356

Merged
TheWitness merged 5 commits intodevelopfrom
workflow
Feb 6, 2026
Merged

Updating workflow to include code validation PHP8.4 and MariaDB 10.6#356
TheWitness merged 5 commits intodevelopfrom
workflow

Conversation

@TheWitness
Copy link
Member

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the CI workflow to improve code validation and testing coverage for the Intropage plugin. It adds support for PHP 8.4, migrates from MySQL 8.0 to MariaDB 11.8, adds comprehensive code quality checks (linting and coding standards), and improves the integration testing approach by running Cacti's main poller instead of just the plugin's poller.

Changes:

  • Expands PHP version testing matrix to include PHP 8.4
  • Switches database service from MySQL 8.0 to MariaDB 11.8
  • Adds linting and PHP CS Fixer validation steps
  • Updates poller testing to run Cacti's main poller with success validation
  • Standardizes naming from "Intropage" to "intropage" in step names for technical consistency
  • Removes unnecessary whitespace for cleaner workflow file

@TheWitness TheWitness changed the title Updating workflow to include Cacti code validation PHP8.4 and MariaDB 11.8 Updating workflow to include Cacti code validation PHP8.4 and MariaDB 10.6 Feb 5, 2026
@TheWitness TheWitness changed the title Updating workflow to include Cacti code validation PHP8.4 and MariaDB 10.6 Updating workflow to include code validation PHP8.4 and MariaDB 10.6 Feb 5, 2026
@TheWitness TheWitness merged commit 346a9c4 into develop Feb 6, 2026
4 checks passed
@TheWitness TheWitness deleted the workflow branch February 15, 2026 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants